Skip to content

[PB-4450]: refactor/extract Share Dialog logic to custom hooks #1819

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
xabg2 merged 15 commits into from
Feb 11, 2026
Merged

[PB-4450]: refactor/extract Share Dialog logic to custom hooks #1819

xabg2 merged 15 commits into from
Feb 11, 2026

Conversation

@xabg2
Copy link
Contributor

@xabg2 xabg2 commented Jan 20, 2026 •
edited
Loading

Description

Extracting the logic from the Shared Dialog to custom hooks.

Related Issues

Related Pull Requests

Checklist

  • Changes have been tested locally.
  • Unit tests have been written or updated as necessary.
  • The code adheres to the repository's coding standards.
  • Relevant documentation has been added or updated.
  • No new warnings or errors have been introduced.
  • SonarCloud issues have been reviewed and addressed.
  • QA Passed

Testing Process

Additional Notes

@xabg2 xabg2 self-assigned this Jan 20, 2026
@xabg2 xabg2 added the enhancement New feature or request label Jan 20, 2026
@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Jan 20, 2026 •
edited
Loading

Deploying drive-web with  Cloudflare Pages  Cloudflare Pages

Latest commit: 56aae94
Status: ✅  Deploy successful!
Preview URL: https://bfc6a845.drive-web.pages.dev
Branch Preview URL: https://refactor-share-dialog-logic.drive-web.pages.dev

View logs

Base automatically changed from refactor/share-dialog-components to master January 21, 2026 10:16
@xabg2 xabg2 changed the base branch from master to feature/share-dialog-custom-hooks January 23, 2026 09:07
Base automatically changed from feature/share-dialog-custom-hooks to master January 27, 2026 11:14
@xabg2 xabg2 marked this pull request as ready for review January 27, 2026 13:22
@sonarqubecloud
Copy link

sonarqubecloud bot commented Feb 11, 2026

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@xabg2 xabg2 merged commit fa4c35c into master Feb 11, 2026
10 checks passed
@xabg2 xabg2 deleted the refactor/share-dialog-logic branch February 11, 2026 10:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@larryrider larryrider larryrider approved these changes

@CandelR CandelR CandelR approved these changes

Assignees

@xabg2 xabg2

Labels

enhancement New feature or request

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@xabg2 @larryrider @CandelR